csskeyframes: Split new() into alloc() and new()

The new() function is supposed to return an empty fully initialized
GtkCssKeyframes object, while the alloc() function just allocates and
initializes static values. So alloc() can be used for copying or
resolving keyframes.

Fixes a memleak when resolving keyframes.

diff --git a/gtk/gtkcsskeyframes.c b/gtk/gtkcsskeyframes.c
index f25b0c4100aca3c1c90c85d26bd74c141f7e472d..5fe905593d470f67a7c8e14e035a3039cc706112 100644 (file)
--- a/gtk/gtkcsskeyframes.c
+++ b/gtk/gtkcsskeyframes.c
@@ -170,13 +170,23 @@ gtk_css_keyframes_lookup_property (GtkCssKeyframes *keyframes,
 }
 
 static GtkCssKeyframes *
-gtk_css_keyframes_new (void)
+gtk_css_keyframes_alloc (void)
 {
   GtkCssKeyframes *keyframes;
 
   keyframes = g_slice_new0 (GtkCssKeyframes);
   keyframes->ref_count = 1;
 
+  return keyframes;
+}
+
+static GtkCssKeyframes *
+gtk_css_keyframes_new (void)
+{
+  GtkCssKeyframes *keyframes;
+
+  keyframes = gtk_css_keyframes_alloc ();
+
   gtk_css_keyframes_add_keyframe (keyframes, 0);
   gtk_css_keyframes_add_keyframe (keyframes, 1);
 
@@ -439,7 +449,7 @@ _gtk_css_keyframes_compute (GtkCssKeyframes         *keyframes,
   g_return_val_if_fail (GTK_IS_CSS_COMPUTED_VALUES (values), NULL);
   g_return_val_if_fail (parent_values == NULL || GTK_IS_CSS_COMPUTED_VALUES (parent_values), NULL);
 
-  resolved = gtk_css_keyframes_new ();
+  resolved = gtk_css_keyframes_alloc ();
   resolved->n_keyframes = keyframes->n_keyframes;
   resolved->keyframe_progress = g_memdup (keyframes->keyframe_progress, keyframes->n_keyframes * sizeof (double));
   resolved->n_properties = keyframes->n_properties;